-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: move to gh actions #85
Conversation
c00ce80
to
2ac79c3
Compare
d1aa699
to
954cd2a
Compare
ed96922
to
4b59b4b
Compare
.github/workflows/ci.yaml
Outdated
branches: | ||
- main | ||
- release-* | ||
schedule: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does the schedule mean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean not sure which step runs there
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
schedule is cron, runs weekly the reproducibilty, there are if
condition to match things
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that reproducibility depends on label, not on schedule event?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ahh missed it
.github/workflows/ci.yaml
Outdated
env: | ||
CI_ARGS: --cache-from=type=registry,ref=registry.dev.siderolabs.io/${GITHUB_REPOSITORY}:buildcache --cache-to=type=registry,ref=registry.dev.siderolabs.io/${GITHUB_REPOSITORY}:buildcache,mode=max | ||
pull_request: | ||
types: [opened, synchronize, reopened, labeled] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does it mean it will run the build each time labeled
is touched?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only if label is removed and added
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, but technically on any label change? it will re-run all steps?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeh, kind of, but the step will be skipped if label is not matched
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but the build will still run though... I wonder if it should be a separate workflow?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeh, even if separate workflow, it starts a job anyways, so not much different. Well i could have a different one that just triggers on label and skip at job level. I guess it's probably easier to read
.github/workflows/ci.yaml
Outdated
if: github.event_name != 'schedule' | ||
run: | | ||
make | ||
- name: reproducibility |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does it mean it runs after build
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, only on schedule or when there's the label
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, but should it be a separate workflow? would it be cleaner?
this will have its own status, as reproducibility
is independent of all other steps
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeh,i initially had that, felt like a lot of duplication, so moved it along the same job
6de4ea8
to
d54ac0d
Compare
Move to GitHub actions. Signed-off-by: Noel Georgi <[email protected]>
d54ac0d
to
639ddad
Compare
/m |
Move to GitHub actions.